Skip to content

add mypy check#42

Merged
chrishavlin merged 1 commit intodata-exp-lab:mainfrom
chrishavlin:enable_typechecking_ci
Apr 1, 2025
Merged

add mypy check#42
chrishavlin merged 1 commit intodata-exp-lab:mainfrom
chrishavlin:enable_typechecking_ci

Conversation

@chrishavlin
Copy link
Copy Markdown
Collaborator

@chrishavlin chrishavlin commented Mar 31, 2025

in progress: still some failures (22 locally)

This adds a mypy check to CI and fixes a bunch of type hints. Pretty minimal changes to the code were necessary: just a bit of refactoring to make the typing simpler.

@chrishavlin chrishavlin added enhancement New feature or request ci_testing CI and testing related fixes and improvements labels Mar 31, 2025
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 31, 2025

Codecov Report

Attention: Patch coverage is 99.27536% with 1 line in your changes missing coverage. Please review.

Project coverage is 98.28%. Comparing base (18abe5f) to head (918973b).
Report is 22 commits behind head on main.

Files with missing lines Patch % Lines
inheritance_explorer/inheritance_explorer.py 98.07% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #42      +/-   ##
==========================================
+ Coverage   97.43%   98.28%   +0.85%     
==========================================
  Files          11       11              
  Lines         585      643      +58     
==========================================
+ Hits          570      632      +62     
+ Misses         15       11       -4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@chrishavlin chrishavlin force-pushed the enable_typechecking_ci branch from 0bcb85e to 918973b Compare April 1, 2025 20:03
@chrishavlin chrishavlin added this to the 0.2.1 milestone Apr 1, 2025
@chrishavlin chrishavlin merged commit ecde164 into data-exp-lab:main Apr 1, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci_testing CI and testing related fixes and improvements enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant